Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GTK3] Do not crash in SetData event #1612

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

basilevs
Copy link
Contributor

@basilevs basilevs commented Nov 22, 2024

The crash happens when renderers are replaced during render. If replacement is postponed, the crash no longer happens. The failure only happens in Ubuntu 24 (GTK 3.24.41).

Fixes #678.
Is tested by #1611.

Copy link
Contributor

github-actions bot commented Nov 22, 2024

Test Results

   483 files  ±0     483 suites  ±0   9m 13s ⏱️ +35s
 4 095 tests ±0   4 085 ✅ ±0   7 💤 ±0  3 ❌ ±0 
16 173 runs  ±0  16 080 ✅ ±0  90 💤 ±0  3 ❌ ±0 

For more details on these failures, see this check.

Results for commit 09cfb68. ± Comparison against base commit 266cc26.

♻️ This comment has been updated with latest results.

@basilevs
Copy link
Contributor Author

basilevs commented Nov 22, 2024

Test failures are caused by #1564

@basilevs basilevs marked this pull request as ready for review November 22, 2024 15:23
@basilevs basilevs force-pushed the issue_678_setdata_fix branch 2 times, most recently from 7c58564 to b1bf254 Compare November 25, 2024 11:28
GTK 3.24.41 (Ubuntu 24) crashes when renderers are replaced during
render.
If replacement is postponed, the crash no longer happens.

Fixes eclipse-platform#678.
@eselmeister
Copy link

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SIGSEGV in Tree.cellDataProc when calling TreeItem.setImage
2 participants